Add file upload support for chat sessions (#19) by GunaPalanivel · Pull Request #61 · jenkinsci/resources-ai-chatbot-plugin

GunaPalanivel · 2025-12-17T13:52:16Z

Summary

Adds the ability for users to attach files when sending messages to the chatbot, giving the LLM richer context for troubleshooting Jenkins issues.

Changes

Backend (Python/FastAPI)

New file_service.py module for file processing and validation
Added multipart form upload endpoint POST /sessions/{session_id}/message/upload
Added GET /files/supported-extensions endpoint for frontend validation
Updated chat_service.py to incorporate file content into LLM prompts
New Pydantic models: FileAttachment, ChatRequestWithFiles, SupportedExtensionsResponse

Frontend (React/TypeScript)

Added attach button (📎) to the input component
File preview chips showing name, size, and remove option
Image thumbnails and file icons displayed in chat messages
Client-side validation for file size and extension

Supported File Types

Logs/Text: .txt, .log, .md
Code: .py, .js, .ts, .tsx, .java, .groovy, .sh
Config: .json, .xml, .yaml, .yml, .properties, .toml
Jenkins: Jenkinsfile, .groovy
Images: .png, .jpg, .jpeg, .gif, .webp, .bmp

Limits

Max file size: 10MB per file
Images are base64 encoded for LLM vision capabilities

Testing

Added unit tests for file service (test_file_service.py)
Added route tests for upload endpoint (test_file_upload.py)
Updated frontend tests for Input component

How to Test

Start the backend and frontend servers
Click the 📎 button or drag files into the input area
Attach a log file or screenshot and ask a question
Verify the file content appears in the chat and the LLM references it

Closes #19

- Add file_service.py for processing uploaded files (text extraction, validation) - Support 30+ file types including logs, code files, configs, and images - Add multipart upload endpoint POST /sessions/{id}/message/upload - Add GET /files/supported-extensions endpoint - Update frontend with attach button, file preview chips, and drag-drop ready UI - Display image thumbnails and file icons in chat messages - Add file size validation (10MB limit) and extension checks - Include unit tests for file service and upload routes

cnu1812 · 2025-12-17T15:29:51Z

Also, a few checks are failing. Take a note!

- Add python-multipart dependency for form data support - Use FileType Enum instead of string for type field (schemas.py) - Add proper model_validator for message/files validation (schemas.py) - Use XML-style tags for robust file context formatting (file_service.py) - Handle hidden files (.env, .gitignore, etc.) as text files - Add 'raise ... from e' pattern for exception chaining (chatbot.py) - Close uploaded files in finally block (chatbot.py) - Remove trailing whitespace and fix line length issues - Remove unused imports (Optional, pytest, TestClient) - Add tests for file size limit enforcement - Add tests for text truncation - Add tests for hidden files support - Add tests for file input trigger callback - Add whitespace-only negative case test - Use data-testid instead of emoji for file icons - Add missing mocks in Chatbot.test.tsx (fetchSupportedExtensions, etc.)

GunaPalanivel · 2025-12-17T16:02:04Z

Also, a few checks are failing. Take a note!

CI Failures Fixed

Backend Tests Failure

Form data requires "python-multipart" to be installed.

Resolution: Added python-multipart==0.0.20 to requirements.txt

Frontend Tests Failure

TypeError: (0 , chatbot_1.fetchSupportedExtensions) is not a function

Resolution: Added missing mocks in Chatbot.test.tsx:

jest.mock("../api/chatbot", () => ({
  fetchChatbotReply: jest.fn()...,
  fetchChatbotReplyWithFiles: jest.fn()...,
  fetchSupportedExtensions: jest.fn().mockResolvedValue({...}),
  validateFile: jest.fn().mockReturnValue({ isValid: true }),
  fileToAttachment: jest.fn()...,
  // ... other mocks
}));

Pylint Failures

Fixed all pylint issues:

Trailing whitespace (C0303)
Line too long (C0301)
Unused imports (W0611)
raise-missing-from (W0707)
Unnecessary pass statement (W0107)
Wrong import order (C0411)

…orrect data - Added mock_get_chatbot_reply.assert_called_once() assertions - Verify filename and content are correctly passed to service - Check multiple files are all processed correctly - Verify truncation logic works by checking content length

…orrect data

berviantoleo

Please see my comments. Some of them are required changes to strengthen the validation.

- Add python-magic for MIME type detection from content - Implement detect_mime_type_from_content() with fallback signatures - Add validate_file_content_type() to verify content matches extension - Block executables disguised as text files - Reject images with mismatched content/extension - Add comprehensive unit tests for new validation functions

…llback

berviantoleo · 2025-12-19T04:37:03Z

Please have a look at the failed CIs

…matting - Prioritize our magic byte signatures over python-magic for images - This ensures consistent behavior across different environments - Fix pylint unused-argument warning in test_file_upload.py - Fix broad-exception-caught by catching specific exceptions - Run prettier to fix frontend formatting issues - Update test for unknown content detection

GunaPalanivel · 2025-12-22T16:10:19Z

Hey @berviantoleo, thanks for the review! We’ve been hitting some stubborn CI fails—mostly due to PyTorch and CUDA version conflicts on the latest Python. Our plan is to relax the strict version pins and let pip resolve the right dependencies, following PyTorch’s official guidance. If you’re good with this approach, we’ll go ahead and update the requirements accordingly. Let us know your thoughts!

berviantoleo · 2025-12-22T23:06:46Z

Hey @berviantoleo, thanks for the review! We’ve been hitting some stubborn CI fails—mostly due to PyTorch and CUDA version conflicts on the latest Python. Our plan is to relax the strict version pins and let pip resolve the right dependencies, following PyTorch’s official guidance. If you’re good with this approach, we’ll go ahead and update the requirements accordingly. Let us know your thoughts!

Good to go. However, @GunaPalanivel, please focusing to fix 3.12 and 3.13. I'm just experimenting with 3.14, so you can ignore the install dependencies error at the moment.

GunaPalanivel · 2025-12-24T04:00:01Z

@berviantoleo Done.

GunaPalanivel requested a review from a team as a code owner December 17, 2025 13:52

cnu1812 requested changes Dec 17, 2025

View reviewed changes

GunaPalanivel added 2 commits December 17, 2025 21:37

test(jenkinsci#19): Add assertions to verify service layer receives c…

f31876f

…orrect data

GunaPalanivel requested a review from cnu1812 December 17, 2025 16:24

berviantoleo reviewed Dec 18, 2025

View reviewed changes

Comment thread chatbot-core/api/services/file_service.py

berviantoleo reviewed Dec 18, 2025

View reviewed changes

Comment thread chatbot-core/api/services/file_service.py

berviantoleo reviewed Dec 18, 2025

View reviewed changes

Comment thread chatbot-core/api/services/file_service.py Outdated

berviantoleo requested changes Dec 18, 2025

View reviewed changes

GunaPalanivel added 2 commits December 18, 2025 15:20

fix(jenkinsci#19): Reject images with unknown MIME type instead of fa…

2543b0b

…llback

GunaPalanivel requested a review from berviantoleo December 18, 2025 11:34

GunaPalanivel and others added 2 commits December 19, 2025 12:12

Merge branch 'main' into feature/19-file-upload-support

029ed2e

berviantoleo approved these changes Dec 22, 2025

View reviewed changes

berviantoleo added the enhancement For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted label Dec 22, 2025

Merge branch 'main' into feature/19-file-upload-support

d163334

GunaPalanivel force-pushed the feature/19-file-upload-support branch from d8bf116 to d163334 Compare December 22, 2025 14:01

GunaPalanivel requested a review from berviantoleo December 22, 2025 16:10

GunaPalanivel force-pushed the feature/19-file-upload-support branch from 88a97e8 to d163334 Compare December 24, 2025 14:09

berviantoleo approved these changes Dec 24, 2025

View reviewed changes

cnu1812 merged commit 1fc36ef into jenkinsci:main Dec 26, 2025
34 of 37 checks passed

Yugansh5013 mentioned this pull request Jan 23, 2026

docs: gsoc report update #118

Merged

JayDS22 mentioned this pull request Mar 29, 2026

docs: add missing endpoint docs to api.md #320

Merged

Uh oh!

Conversation

GunaPalanivel commented Dec 17, 2025

Summary

Changes

Backend (Python/FastAPI)

Frontend (React/TypeScript)

Supported File Types

Limits

Testing

How to Test

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cnu1812 commented Dec 17, 2025

Uh oh!

GunaPalanivel commented Dec 17, 2025

CI Failures Fixed

Backend Tests Failure

Frontend Tests Failure

Pylint Failures

Uh oh!

Uh oh!

Uh oh!

Uh oh!

berviantoleo left a comment

Choose a reason for hiding this comment

Uh oh!

berviantoleo commented Dec 19, 2025

Uh oh!

GunaPalanivel commented Dec 22, 2025

Uh oh!

berviantoleo commented Dec 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

GunaPalanivel commented Dec 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

berviantoleo commented Dec 22, 2025 •

edited

Loading

GunaPalanivel commented Dec 24, 2025 •

edited

Loading